Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

update variables files for gke nodepool taints #1358

Merged
merged 4 commits into from
May 5, 2023

Conversation

jackspyder
Copy link
Contributor

Hello,

A small fix to the gke blueprint, and the fast gke module to allow passing of taint objects down through the modules.

@juliocc
Copy link
Collaborator

juliocc commented May 5, 2023

Thanks for the fix!

You have to run the tfdoc script to update the README file.

Copy link
Collaborator

@ludoo ludoo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for spotting this!

@jackspyder
Copy link
Contributor Author

Just noticed the readme issue, getting setup and will add my commit with the changes shortly.

@jackspyder
Copy link
Contributor Author

@ludoo @juliocc docs fixed, sorry i missed them!

@ludoo
Copy link
Collaborator

ludoo commented May 5, 2023

You should see a merge button, just go a ahead and merge. :)

@jackspyder
Copy link
Contributor Author

You should see a merge button, just go a ahead and merge. :)

You should see a merge button, just go a ahead and merge. :)

No merge button sorry, unless im being extremely dense.

@ludoo
Copy link
Collaborator

ludoo commented May 5, 2023

ach, my fault... let me merge, sorry for that and thanks for the PR

@ludoo ludoo merged commit 491b52f into GoogleCloudPlatform:master May 5, 2023
lcaggio pushed a commit that referenced this pull request May 5, 2023
* update variables files for gke node config taints to allow passing of node objects

* forgot to run terraform fmt..

* update module docs
@jackspyder jackspyder deleted the jackspyder/gke-taints-fix branch May 8, 2023 18:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants